Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[16.0] [MIG] pos_reset_search #1030

Merged
merged 26 commits into from
Dec 19, 2023
Merged

[16.0] [MIG] pos_reset_search #1030

merged 26 commits into from
Dec 19, 2023

Conversation

victor-champonnois
Copy link
Member

@victor-champonnois victor-champonnois commented Aug 2, 2023 •
edited
Loading

Currently not working. When searching and then clicking on a product, an error is raised :

TypeError: ProductScreenChildren[key].el is undefined
    _clickProduct@http://localhost:8012/web/assets/debug/point_of_sale.assets.js:45187:25 (/pos_reset_search/static/src/js/ProductScreen.js:13)
    __trigger@http://localhost:8012/web/assets/debug/web.assets_common.js:22576:21 (/web/static/src/legacy/legacy_component.js:89)
    trigger@http://localhost:8012/web/assets/debug/web.assets_common.js:22560:14 (/web/static/src/legacy/legacy_component.js:73)
    template/hdlr2<@http://localhost:8012/web/assets/debug/web.assets_common.js line 21902 > Function:14:25 (/web/static/lib/owl/owl.js:5508)
    mainEventHandler@http://localhost:8012/web/assets/debug/web.assets_common.js:22169:25 (/web/static/lib/owl/owl.js:5775)
    listener@http://localhost:8012/web/assets/debug/web.assets_common.js:16792:20 (/web/static/lib/owl/owl.js:398)

So far, I don't understand the problem. Help would be very welcomed :)

Fkawala and others added 25 commits August 2, 2023 15:54
@Fkawala @victor-champonnois
Reset product search when product is selected with mouse.
916289e
@Fkawala @victor-champonnois
fix typo.
357ec25
@victor-champonnois
Lint fix + readme.
76ec2b4
@victor-champonnois
fix website in manifest.
8c31930
@Fkawala @victor-champonnois
Built README.rst with oca-gen-addon-readme + standard icon.
82c27d8
@Fkawala @victor-champonnois
Built README.rst with oca-gen-addon-readme + standard icon. contd.
d94b310
@Fkawala @victor-champonnois
Fix ._super call.
b3860e9
@Fkawala @victor-champonnois
Added myself as maintainer.
32160d9
@oca-travis @victor-champonnois
[UPD] Update pos_reset_search.pot
01377ea
@Fkawala @victor-champonnois
Set maintainers for three modules.
4bb88ee
@OCA-git-bot @victor-champonnois
[UPD] README.rst
e54fb3c
@Fkawala @victor-champonnois
Migration to v12.
c540bdb
@oca-travis @victor-champonnois
[UPD] Update pos_reset_search.pot
777cc2c
@OCA-git-bot @victor-champonnois
[UPD] README.rst
d88600b
@danimv5 @victor-champonnois
Added translation using Weblate (Spanish)
fd2fdaf
@dsolanki-initos @victor-champonnois
[IMP] pos_reset_search: isort, black, prettier.
cb8c35e
@dsolanki-initos @victor-champonnois
[MIG] Migrate module pos_reset_search to v14.
c51767f
@oca-travis @victor-champonnois
[UPD] Update pos_reset_search.pot
41a49fc
@OCA-git-bot @victor-champonnois
[UPD] README.rst
24ad966
@oca-transbot @victor-champonnois
Update translation files
e543133 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: pos-14.0/pos-14.0-pos_reset_search
Translate-URL: https://translation.odoo-community.org/projects/pos-14-0/pos-14-0-pos_reset_search/
@rousseldenis @victor-champonnois
[14.0][FIX] pos_reset_search: Iterate on owl children
8e76265
@OCA-git-bot @victor-champonnois
pos_reset_search 14.0.1.0.1
5c1ebf2
@mymage @victor-champonnois
Added translation using Weblate (Italian)
65e36a1
@victor-champonnois
[IMP] pos_reset_search: black, isort, prettier
9ad2370
@victor-champonnois
[MIG] pos_reset_search: Migration to 15.0
c592397
@legalsylvain
Copy link
Contributor

/ocabot migration pos_reset_search

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Sep 26, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Sep 26, 2023
Open
37 tasks
@victor-champonnois @carmenbianca
[MIG] pos_reset_search: Migration to 16.0
c3e205e 
Co-Authored-By: Carmen Bianca BAKKER <carmen@coopiteasy.be>
@carmenbianca
Copy link
Member

@victor-champonnois I fixed this PR for ya.

@robinkeunen
Copy link

@victor-champonnois Can you undraft this PR ?
@legalsylvain this is waiting a Christmas review 🥳

@victor-champonnois victor-champonnois marked this pull request as ready for review December 19, 2023 17:01
@chienandalu
Copy link
Member

🎅

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-1030-by-chienandalu-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 22b8663 into OCA:16.0 Dec 19, 2023
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at b26c7a2. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@carmenbianca carmenbianca carmenbianca approved these changes

@legalsylvain legalsylvain legalsylvain approved these changes

Assignees
No one assigned
Labels
approved merged 🎉
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.